Migrate sonic-mgmt-framework container to Debian Trixie#26548
Conversation
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Signed-off-by: Amruta Sali <amruta_sali@dell.com>
75b8c91 to
8cd8fab
Compare
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
@saiarcot895 Please add this to tracker - sonic-net/SONiC#2169 . |
There was a problem hiding this comment.
Pull request overview
This PR migrates the docker-sonic-mgmt-framework container to build on the Debian Trixie base (via docker-config-engine-trixie), aligning the container’s build/runtime environment with the newer BLDENV=trixie build flow.
Changes:
- Switch
docker-sonic-mgmt-frameworkbuild dependencies/load base fromdocker-config-engine-bookwormtodocker-config-engine-trixie. - Update the container Dockerfile base image to
docker-config-engine-trixie-*and adjust installed Debian packages accordingly.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| rules/docker-sonic-mgmt-framework.mk | Moves the mgmt-framework docker image’s base/load/debug dependency wiring from bookworm config-engine to trixie config-engine, and reclassifies it under “trixie” docker lists. |
| dockers/docker-sonic-mgmt-framework/Dockerfile.j2 | Switches the Docker build ARG base image to docker-config-engine-trixie-* and updates apt dependencies for Trixie. |
@amrutasali Please note #26499. |
@saiarcot895 yes I'm aware and got it confirmed - #25959 (comment) .So will just bump Golang version for soni-gnmi repository. |
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
…mpiled against cjson headers inside the container Signed-off-by: Amruta Sali <amruta_sali@dell.com>
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run Azure.sonic-buildimage |
|
/azpw ms_conflict |
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azpw ms_conflict |
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azpw ms_conflict |
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
…o 1.24.4 (trixie-only) and are only used by trixie-specific docker images. Add BLDENV=trixie conditionals following the libpcre3 pattern to avoid bookworm build failures. Signed-off-by: Amruta Sali <amruta_sali@dell.com>
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
@saiarcot895 this is ready to be merged |
|
@yxieca ping for merge. This blocks sonic-mgmt-common submodule update, which in turn blocks sonic-gnmi submodule (already 3 weeks old). @saiarcot895 is this PR good for merge? |
…equire Go 1.24.4 (trixie-only) and are only used by trixie-specific docker images. Add BLDENV=trixie conditionals following the libpcre3 pattern to avoid bookworm build failures." This reverts commit 9585d23. Signed-off-by: Amruta Sali <amruta_sali@dell.com>
|
/azp run Azure.sonic-buildimage |
|
Azure Pipelines successfully started running 1 pipeline(s). |
) What: Migrated docker-sonic-mgmt-framework container from Debian Bookworm to Trixie. Why: Required for the Trixie migration effort (sonic-net/SONiC#2169). How: Updated Docker build files for Trixie. Updated dependency packages accordingly. Dependencies: sonic-mgmt-common/pull/211 and sonic-mgmt-framework/pull/158 with submodule updates. Testing: Container builds successfully under BLDENV=trixie. Verified docker OS shows Trixie and basic curl test on REST server works. All CI builds and KVM tests passed. Signed-off-by: Amruta Sali <amruta_sali@dell.com> Signed-off-by: mhchann <mhchann082@gmail.com>
Why I did it - Required for sonic-net/SONiC#2169
How I did it - Updated the Docker build files to build for Debian Trixie. The dependency packages are updated accordingly.
How to verify it - The sonic-mgmt-framework container builds successfully under BLDENV=trixie. Installed it on a recent SONiC image and verified that the docker os distribution shows Trixie and basic curl test on REST server hosted within the docker works fine.
Note: The sonic-mgmt-framework build for trixie has a dependency on below PRs and then a submodule update for the 2 repos :
sonic-net/sonic-mgmt-common#211
sonic-net/sonic-mgmt-framework#158